Ab#78611 #302

spbsoluble · 2025-12-08T22:10:27Z

No description provided.

Signed-off-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

…ntegration-manifest.json feat(cli/pam): Add support for get and delete pam types. fix(cli/pam): Remove check for bug ab#63171 Signed-off-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

Signed-off-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

…ile for PAM types. refactor(cli/pam): Split PAM code into 2 files one for PAM Types and the other for PAM providers. Signed-off-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

…avoid storing creds on disk and in env vars. Signed-off-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

Signed-off-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

… passed Signed-off-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

Signed-off-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

BREAKING CHANGE: All `pam types-*` are now `pam-types <action>` Signed-off-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

Signed-off-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

…e changes for a profile. Signed-off-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

Signed-off-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

…some of the definition issues. Signed-off-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

Signed-off-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

…t --csv` Signed-off-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

Copilot

Pull request overview

Copilot reviewed 118 out of 120 changed files in this pull request and generated no new comments.

Comments suppressed due to low confidence (3)

pkg/version/version.go:1

The build date is set to December 4, 2025, which is in the future relative to the current date (December 15, 2025 according to the context, but code was likely generated earlier). Build dates should typically reflect when the code was actually built, not future dates.
cmd/storesBulkOperations.go:1
The nested property handling duplicates the type assertion prop.(map[string]interface{}) on line 827 after checking it on line 826. Consider storing the type assertion result in a variable to avoid duplication and improve code clarity.
cmd/pam_types.json:1
Corrected spelling of 'compatability' to 'compatibility'.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

…passed. Signed-off-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

…ers for secrets. Signed-off-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

Copilot

Pull request overview

Copilot reviewed 118 out of 120 changed files in this pull request and generated no new comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

… a PAM provider. fix(cli): `stores import csv` properly handles "ServerUsername" and "ServerPassword" properties. Signed-off-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

Signed-off-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

Copilot

Copilot reviewed 118 out of 120 changed files in this pull request and generated no new comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Signed-off-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

Copilot

Pull request overview

Copilot reviewed 118 out of 120 changed files in this pull request and generated no new comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Signed-off-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

…nds and indicate deprecation Signed-off-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

Signed-off-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

Copilot

Pull request overview

Copilot reviewed 118 out of 120 changed files in this pull request and generated 13 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2026-01-26T20:25:47Z

.github/workflows/mock_tests.yml

+      - name: Set up Go
+        uses: actions/setup-go@v5
+        with:
+          go-version: '1.24'
+          cache: true


This workflow sets up Go 1.24, but go.mod now declares go 1.25. On runners with Go 1.24, go test will fail with a "go.mod requires go >= 1.25" style error. Update the workflow to use Go 1.25 (or lower the module go version if 1.25 isn't required).

Copilot · 2026-01-26T20:25:47Z

.github/workflows/mock_tests.yml

+      - name: Set up Go
+        uses: actions/setup-go@v5
+        with:
+          go-version: '1.24'
+          cache: true


This workflow sets up Go 1.24, but go.mod now declares go 1.25. On runners with Go 1.24, go test will fail with a "go.mod requires go >= 1.25" style error. Update the workflow to use Go 1.25 (or lower the module go version if 1.25 isn't required).

Copilot · 2026-01-26T20:25:47Z

.github/workflows/mock_tests.yml

+      - name: Set up Go
+        uses: actions/setup-go@v5
+        with:
+          go-version: '1.24'
+          cache: true


This workflow sets up Go 1.24, but go.mod now declares go 1.25. On runners with Go 1.24, go test will fail with a "go.mod requires go >= 1.25" style error. Update the workflow to use Go 1.25 (or lower the module go version if 1.25 isn't required).

Copilot · 2026-01-26T20:25:48Z

cmd/pam_test.go

 	testCmd := RootCmd
-	testCmd.SetArgs([]string{"pam", "--help"})
+	testCmd.SetArgs([]string{"pam-types", "--help"})
 	err := testCmd.Execute()


These tests are named/structured around the pam command (PAM Provider APIs), but the updated args call pam-types. The repository still has a separate pam command (see cmd/pam.go) and docs for kfutil pam ... provider operations. This will cause the provider tests to exercise the wrong CLI and likely fail. Switch these args back to pam (or move pam-types-specific tests into pamTypes_test.go).

Copilot · 2026-01-26T20:25:48Z

cmd/pam_test.go

 	testCmd := RootCmd
 	// test
-	testCmd.SetArgs([]string{"pam", "update", "--from-file", updatedFileName})
+	testCmd.SetArgs([]string{"pam-types", "update", "--from-file", updatedFileName})
 	output := captureOutput(


Test_PAMUpdateCmd is updating a PAM provider instance from a JSON file, but it invokes pam-types update. The provider update command is pam update (docs/kfutil_pam_update.md, cmd/pam.go). Switch the args to pam update so the test exercises the correct CLI.

Copilot · 2026-01-26T20:25:49Z

cmd/helpers.go

+		propSecret, isSecret := prop.(map[string]interface{})
+		if isSecret {
+			formattedSecret := map[string]map[string]interface{}{
+				"Value": {},
+			}
+			isManaged := propSecret["IsManaged"].(bool)
+			if isManaged { // managed secret, i.e. PAM Provider in use
+				formattedSecret["Value"] = reformatPamSecretForPost(propSecret)
+			} else {
+				// non-managed secret i.e. a KF-encrypted secret, or no value
+				// still needs to be reformatted to required POST format
+				formattedSecret["Value"] = map[string]interface{}{
+					"SecretValue": propSecret["Value"],
+				}
+			}
+
+			// update Properties object with newly formatted secret, compliant with POST requirements
+			certStore.Properties[propName] = formattedSecret
+		}


formatStoreProperties treats any map[string]interface{} property value as a secret and unconditionally asserts propSecret["IsManaged"].(bool). Store properties can legitimately be JSON objects that are not secrets (the export loop below already handles generic map[string]interface{} properties), which will panic here. Guard this conversion by checking for the expected secret shape/keys (e.g., presence and type of IsManaged and Value/ProviderId) before type asserting, and skip non-secret object properties.

Suggested change

propSecret, isSecret := prop.(map[string]interface{})

if isSecret {

formattedSecret := map[string]map[string]interface{}{

"Value": {},

}

isManaged := propSecret["IsManaged"].(bool)

if isManaged { // managed secret, i.e. PAM Provider in use

formattedSecret["Value"] = reformatPamSecretForPost(propSecret)

} else {

// non-managed secret i.e. a KF-encrypted secret, or no value

// still needs to be reformatted to required POST format

formattedSecret["Value"] = map[string]interface{}{

"SecretValue": propSecret["Value"],

}

}

// update Properties object with newly formatted secret, compliant with POST requirements

certStore.Properties[propName] = formattedSecret

}

propSecret, ok := prop.(map[string]interface{})

if !ok {

// not an object; leave as-is

continue

}

// Guard: ensure this object looks like a secret before asserting types.

isManagedRaw, hasIsManaged := propSecret["IsManaged"]

if !hasIsManaged {

// object without IsManaged is not treated as a secret

continue

}

isManaged, ok := isManagedRaw.(bool)

if !ok {

// unexpected type for IsManaged; skip to avoid panic

continue

}

valueRaw, hasValue := propSecret["Value"]

providerIDRaw, hasProviderID := propSecret["ProviderId"]

// If the object doesn't have either a Value or ProviderId, treat it as a generic object.

if !hasValue && !hasProviderID {

continue

}

formattedSecret := map[string]map[string]interface{}{

"Value": {},

}

if isManaged { // managed secret, i.e. PAM Provider in use

// Only call reformatPamSecretForPost when it looks like a managed secret payload.

if hasProviderID {

_ = providerIDRaw // presence is enough here; type checks are done by reformatPamSecretForPost or downstream.

formattedSecret["Value"] = reformatPamSecretForPost(propSecret)

} else if hasValue {

// Fallback: treat as non-managed-style data if ProviderId is missing.

formattedSecret["Value"] = map[string]interface{}{

"SecretValue": valueRaw,

}

} else {

// Insufficient data to safely reformat; leave property unchanged.

continue

}

} else {

// non-managed secret i.e. a KF-encrypted secret, or no value

// still needs to be reformatted to required POST format

if !hasValue {

// cannot construct SecretValue without a Value; leave as-is

continue

}

formattedSecret["Value"] = map[string]interface{}{

"SecretValue": valueRaw,

}

}

// update Properties object with newly formatted secret, compliant with POST requirements

certStore.Properties[propName] = formattedSecret

Copilot · 2026-01-26T20:25:49Z

cmd/helpers.go

 			fmt.Println(fmt.Sprintf("{\"error\": \"%s\"}", err))
 		} else {
-			fmt.Errorf(fmt.Sprintf("Fatal error: %s", err))
+			fmt.Errorf("fatal error: %s", err)


In the non-JSON fatal branch, fmt.Errorf("fatal error: %s", err) creates an error that is immediately discarded, so the "fatal" prefix is never surfaced. If you want a distinct fatal message, print/log it (or return the error to the caller) instead of constructing it and ignoring the result.

Suggested change

fmt.Errorf("fatal error: %s", err)

log.Error().Err(err).Msg("fatal error")

Copilot · 2026-01-26T20:25:49Z

cmd/pam_test.go

 		"Listing PAM provider instances", func(t *testing.T) {
 			testCmd := RootCmd
 			// test
-			testCmd.SetArgs([]string{"pam", "list"})
+			testCmd.SetArgs([]string{"pam-types", "list"})
 			output = captureOutput(


testListPamProviders is validating PAM provider instances (expects ProviderType, ProviderTypeParams, etc.), but it invokes pam-types list. pam-types is a different command group for provider types (cmd/pamTypes.go), while provider instances are under pam list. Update the command args to use pam list here.

Copilot · 2026-01-26T20:25:49Z

cmd/pam_test.go

 			// test
 			idInt := int(providerConfig["Id"].(float64))
 			idStr := strconv.Itoa(idInt)
-			testCmd.SetArgs([]string{"pam", "get", "--id", idStr})
+			testCmd.SetArgs([]string{"pam-types", "get", "--id", idStr})
 			output := captureOutput(


This test builds a provider instance list, then uses provider instance IDs with pam-types get. Provider instances are retrieved via pam get (see docs/kfutil_pam_get.md and cmd/pam.go). pam-types get is for provider types. Update the args to call pam get when fetching a provider instance.

Copilot · 2026-01-26T20:25:49Z

cmd/pam_test.go

 			testCmd := RootCmd

-			args := []string{"pam", "create", "--from-file", fileName}
+			args := []string{"pam-types", "create", "--from-file", fileName}
 			// log the args as a string


testCreatePamProvider is creating a PAM provider instance from a JSON file (expects ProviderType etc.), but it invokes pam-types create. Provider instances are created via pam create (docs/kfutil_pam_create.md, cmd/pam.go). Update the args to use pam create (and keep pam-types coverage in pamTypes_test.go).

Signed-off-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

spbsoluble added 30 commits December 4, 2025 10:03

chore(deps): Update dependencies to latest versions

168bfd4

Signed-off-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

feat(cli/pam): Add support for creating a pam provider type from an i…

2be1684

…ntegration-manifest.json feat(cli/pam): Add support for get and delete pam types. fix(cli/pam): Remove check for bug ab#63171 Signed-off-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

chore(store-types): Update internal store-type definitions to latest

b6066f1

Signed-off-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

feat(cli/pam-types): Embed existing PAM types

1be6293

Signed-off-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

feat(cli/pam): Add support for parsing an integration-manifest.json f…

a943161

…ile for PAM types. refactor(cli/pam): Split PAM code into 2 files one for PAM Types and the other for PAM providers. Signed-off-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

feat(cli/auth): Allow for interactive auth on a per command basis to …

f258ec1

…avoid storing creds on disk and in env vars. Signed-off-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

chore(docs): Update auth_providers.md with oauth example.

865ddbd

Signed-off-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

chore(deps): Bump keyfactor-go-client

8167ee4

Signed-off-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

chore(deps): Update all deps to latest

58eaa8e

Signed-off-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

fix(cli/auth): Don't prompt on known values when incomplete config is…

116f85c

… passed Signed-off-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

feat(cli/pam): Delete by name supported.

8a1a11b

Signed-off-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

feat(cli/pam-types)!: Rescope pam types CLI actions to pam-types

daee467

BREAKING CHANGE: All `pam types-*` are now `pam-types <action>` Signed-off-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

chore(docs): Regenerate auto docs.

2c81741

Signed-off-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

chore(deps): Bump keyfactor-go-client

174a3b9

Signed-off-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

chore(tests/pam-types): Add mock tests for pam-types sub CLI

9a7c20e

Signed-off-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

chore(lint): Fix go vet issues.

ecd9076

Signed-off-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

fix(cli/login): login will clear out basic/oauth params if auth typ…

d23514e

…e changes for a profile. Signed-off-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

fix(cli/store-types): Sort store-types list case-insensitively

38ecb29

Signed-off-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

chore(tests/store-types): Convert store-types tests to mocks and fix …

e4adc2b

…some of the definition issues. Signed-off-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

chore(tests/store-types): Add mock server to store-types tests.

3b8e753

Signed-off-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

chore(tests/pam-types): Add mock server to pam-types tests.

19ad7c5

Signed-off-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

chore: Update license headers on files.

be62176

Signed-off-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

chore(ci): Add workflow to run mock tests.

79f1bd1

Signed-off-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

chore(ci): Remove "old" tests workflow

5671c97

Signed-off-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

chore(ci/mock_tests): Dynamic summary for mock tests

880d422

Signed-off-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

chore(ci/mock_tests): Always run mocks

dba0898

Signed-off-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

chore(ci/mock_tests): Always run all store-type mocks

98169ab

Signed-off-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

chore(ci): Update workflow ref versions to latest

c3ca543

Signed-off-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

chore(deps): Bump golang version to v1.25

d6f23fb

Signed-off-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

chore(ci): Fix container digest uploads.

4d82a32

Signed-off-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

spbsoluble added 4 commits December 15, 2025 12:14

fix(cli/orchs): Remove --exp flag from the orchs sub CLIs

d45f67b

Signed-off-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

chore(docs): Update CHANGELOG.md

55d2a3a

Signed-off-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

chore(docs): Add quick doc for updating certificate store UOs via CSV.

02fb606

Signed-off-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

chore(cli/stores): Output number of creates and/or updates for `impor…

bcf53b2

…t --csv` Signed-off-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

Copilot AI review requested due to automatic review settings December 15, 2025 21:14

Copilot started reviewing on behalf of spbsoluble December 15, 2025 21:14 View session

Copilot AI reviewed Dec 15, 2025

View reviewed changes

spbsoluble added 2 commits December 17, 2025 09:18

fix(auth): Attempt env vars during interactive auth if empty conf is …

5415ade

…passed. Signed-off-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

fix(stores): export now properly exports stores that use PAM provid…

a75ab62

…ers for secrets. Signed-off-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

Copilot AI review requested due to automatic review settings January 7, 2026 20:02

Copilot started reviewing on behalf of spbsoluble January 7, 2026 20:02 View session

Copilot AI reviewed Jan 7, 2026

View reviewed changes

spbsoluble added 3 commits January 9, 2026 13:35

feat(cli): stores import csv add support for store Password to be…

98ef719

… a PAM provider. fix(cli): `stores import csv` properly handles "ServerUsername" and "ServerPassword" properties. Signed-off-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

feat(cli): pam migrate to handle Keyfactor secrets.

77d047a

Signed-off-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

chore(deps): Bump keyfactor-api-client version

3970592

Signed-off-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

Copilot AI review requested due to automatic review settings January 9, 2026 21:37

Copilot started reviewing on behalf of spbsoluble January 9, 2026 21:38 View session

Copilot AI reviewed Jan 9, 2026

View reviewed changes

fix(cli): Generate header based on data items

48e7cbb

Signed-off-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

Copilot AI review requested due to automatic review settings January 9, 2026 22:33

spbsoluble force-pushed the ab#78611 branch from 4cab2d1 to 48e7cbb Compare January 9, 2026 22:33

Copilot started reviewing on behalf of spbsoluble January 9, 2026 22:33 View session

Copilot AI reviewed Jan 9, 2026

View reviewed changes

spbsoluble added 3 commits January 19, 2026 09:23

fix(cli): Fix POST payload for ServerUsername and ServerPassword

054b0f1

Signed-off-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

fix(cli/pam): Add back the pam types-list, pam types-create comma…

4dc244c

…nds and indicate deprecation Signed-off-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

chore(cli): Remove makedocs command

8d1837a

Signed-off-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

Copilot AI review requested due to automatic review settings January 26, 2026 20:16

Copilot started reviewing on behalf of spbsoluble January 26, 2026 20:16 View session

Copilot AI reviewed Jan 26, 2026

View reviewed changes

chore(cli/store-types): Update embedding store-type definitions

20d3db1

Signed-off-by: spbsoluble <1661003+spbsoluble@users.noreply.github.com>

-		propSecret, isSecret := prop.(map[string]interface{})
-		if isSecret {
-			formattedSecret := map[string]map[string]interface{}{
-				"Value": {},
-			}
-			isManaged := propSecret["IsManaged"].(bool)
-			if isManaged { // managed secret, i.e. PAM Provider in use
-				formattedSecret["Value"] = reformatPamSecretForPost(propSecret)
-			} else {
-				// non-managed secret i.e. a KF-encrypted secret, or no value
-				// still needs to be reformatted to required POST format
-				formattedSecret["Value"] = map[string]interface{}{
-					"SecretValue": propSecret["Value"],
-				}
-			}
-			// update Properties object with newly formatted secret, compliant with POST requirements
-			certStore.Properties[propName] = formattedSecret
-		}
+		propSecret, ok := prop.(map[string]interface{})
+		if !ok {
+			// not an object; leave as-is
+			continue
+		}
+		// Guard: ensure this object looks like a secret before asserting types.
+		isManagedRaw, hasIsManaged := propSecret["IsManaged"]
+		if !hasIsManaged {
+			// object without IsManaged is not treated as a secret
+			continue
+		}
+		isManaged, ok := isManagedRaw.(bool)
+		if !ok {
+			// unexpected type for IsManaged; skip to avoid panic
+			continue
+		}
+		valueRaw, hasValue := propSecret["Value"]
+		providerIDRaw, hasProviderID := propSecret["ProviderId"]
+		// If the object doesn't have either a Value or ProviderId, treat it as a generic object.
+		if !hasValue && !hasProviderID {
+			continue
+		}
+		formattedSecret := map[string]map[string]interface{}{
+			"Value": {},
+		}
+		if isManaged { // managed secret, i.e. PAM Provider in use
+			// Only call reformatPamSecretForPost when it looks like a managed secret payload.
+			if hasProviderID {
+				_ = providerIDRaw // presence is enough here; type checks are done by reformatPamSecretForPost or downstream.
+				formattedSecret["Value"] = reformatPamSecretForPost(propSecret)
+			} else if hasValue {
+				// Fallback: treat as non-managed-style data if ProviderId is missing.
+				formattedSecret["Value"] = map[string]interface{}{
+					"SecretValue": valueRaw,
+				}
+			} else {
+				// Insufficient data to safely reformat; leave property unchanged.
+				continue
+			}
+		} else {
+			// non-managed secret i.e. a KF-encrypted secret, or no value
+			// still needs to be reformatted to required POST format
+			if !hasValue {
+				// cannot construct SecretValue without a Value; leave as-is
+				continue
+			}
+			formattedSecret["Value"] = map[string]interface{}{
+				"SecretValue": valueRaw,
+			}
+		}
+		// update Properties object with newly formatted secret, compliant with POST requirements
+		certStore.Properties[propName] = formattedSecret

	fmt.Errorf("fatal error: %s", err)
	log.Error().Err(err).Msg("fatal error")

Ab#78611 #302

Are you sure you want to change the base?

Ab#78611 #302

Uh oh!

Conversation

spbsoluble commented Dec 8, 2025

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Copilot AI Jan 26, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 26, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 26, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 26, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 26, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 26, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 26, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 26, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 26, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 26, 2026

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants